PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

const client = new OpenVikingClient(
  baseUrl,
  cfg.apiKey,
  cfg.agentId,
  cfg.timeoutMs,
  tenantAccount,
  tenantUser,
  routingDebugLog,
);

PR Code Suggestions ✨

No code suggestions found for the PR.

1. 原本已有 resolved_user_id，其作用是参与本地 identity/space 推导，并用于 routing debug 日志；它的数值来源不是插件配置，而是调用 /api/v1/system/status 的返回值。相关代码包括 getRuntimeIdentity()、emitRoutingDebug() 和 resolveScopeSpace()。

2. 现在 config.userId 已经被明确作为客户端期望的 tenant user，并通过 X-OpenViking-User 发给服务端；但代码里参与本地 identity/space 推导的仍是 /api/v1/system/status 返回值，建议本地推导 identity/space 时优先使用 config.userId

3. 建议保留 /api/v1/system/status 返回值作为 server_reported_user_id 用于诊断，在两者不一致时输出 warning/ERROR。

@jcp0578 已根据这条 review comment 完成修复并推送：c0956866。

更新点：

1. 本地 identity/space 推导优先使用 config.userId（避免 header 与本地路由空间不一致）。
2. 仍保留 /api/v1/system/status 返回 user 作为 server_reported_user_id 用于诊断。
3. 当 config.userId 与 server reported user 不一致时输出 WARNING user identity mismatch（即使 logFindRequests=false 也会告警，不再静默）。

补充：

• 路由 debug 中新增 server_reported_user_id 字段。
• 增加 UT 覆盖：验证 target_uri 改写到 viking://user/<config.userId>/memories，并验证 mismatch warning 输出。

已验证：npm test -- tests/ut/plugin-normal-flow-real-server.test.ts tests/ut/config.test.ts 通过。